Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GH-101/FP-1458: Header Redesign #222 - Mobile Nav Layout 2 #824

Draft
wants to merge 9 commits into
base: task/GH-101-header-redesign
Choose a base branch
from

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Mar 25, 2024

Overview

(I assume) 2nd attmept at mobile nav layout for GH-101.

Related

Changes

Testing

UI

CAVEAT: Font is not always consistent between states and sites.
CAVEAT: The mobile nav menus have not yet been moved ot their new position in the design.
Known Issues:
1. On CMS & Docs, `s-header__menu-backdrop` adds gap in navigation bar.
2. On CMS, mobile navbar opens and closes too slow.
3. The login link is a bit too wide.
4. Code is difficult to follow (but documented).
@wesleyboar wesleyboar added the paused Started but not actively in progress label Mar 25, 2024
@wesleyboar wesleyboar mentioned this pull request Mar 25, 2024
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
paused Started but not actively in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant